Skip to content

[DEBUG][CI]travis: add more build checks - #351

Merged
plbossart merged 1 commit into
thesofproject:topic/sof-devfrom
xiulipan:pr/ciwerror
Dec 5, 2018
Merged

plbossart merged 1 commit into
thesofproject:topic/sof-devfrom
xiulipan:pr/ciwerror

Conversation

@xiulipan

@xiulipan xiulipan commented Dec 3, 2018

Copy link
Copy Markdown

Add Werror and Wall flag to check if there are any warnings.
Add sparse check.

Signed-off-by: Pan Xiuli xiuli.pan@linux.intel.com

@plbossart plbossart left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple of comments below

Comment thread .travis.yml Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe add hdaudio-defconfig?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you mean use this config
scripts/kconfig/merge_config.sh .config <PATH>/kconfig/base-defconfig <PATH>/kconfig/sof-defconfig <PATH>/kconfig/hdaudio-codecs-defconfig

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep. But just for sparse checks.

@xiulipan

xiulipan commented Dec 5, 2018

Copy link
Copy Markdown
Author

@plbossart
Update the config for sparse check.

Add Werror and Wall flag to check if there are any warnings.
Add sparse check.

Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>

@plbossart plbossart left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, thanks!

@plbossart
plbossart merged commit 62e8db9 into thesofproject:topic/sof-dev Dec 5, 2018
@xiulipan
xiulipan deleted the pr/ciwerror branch December 6, 2018 03:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants